Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix NBA Boxscores using the wrong home values #505

Merged
merged 1 commit into from
Oct 16, 2020

Conversation

roclark
Copy link
Owner

@roclark roclark commented Oct 16, 2020

The Boxscores pages on sports-reference.com were changed to include scores by halves and quarters, which retrieved the incorrect values for the home team as it was instead pulling the first-quarter stats for the away team. By using a new index to retrieve values from, the proper stats can be displayed for the home team.

Fixes #498

Signed-Off-By: Robert Clark robdclark@outlook.com

@roclark roclark added the bug Something isn't working label Oct 16, 2020
@roclark roclark added this to the Release 0.6.0 milestone Oct 16, 2020
@roclark roclark self-assigned this Oct 16, 2020
@codecov
Copy link

codecov bot commented Oct 16, 2020

Codecov Report

Merging #505 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #505   +/-   ##
=======================================
  Coverage   99.97%   99.97%           
=======================================
  Files          55       55           
  Lines       13084    13091    +7     
=======================================
+ Hits        13081    13088    +7     
  Misses          3        3           
Impacted Files Coverage Δ
sportsreference/nba/constants.py 100.00% <ø> (ø)
sportsreference/nba/boxscore.py 100.00% <100.00%> (ø)
sportsreference/utils.py 97.29% <100.00%> (+0.19%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8207956...95384d0. Read the comment docs.

The Boxscores pages on sports-reference.com were changed to include
scores by halves and quarters, which retrieved the incorrect values for
the home team as it was instead pulling the first-quarter stats for the
away team. By using a new index to retrieve values from, the proper
stats can be displayed for the home team.

Signed-Off-By: Robert Clark <robdclark@outlook.com>
@roclark roclark added the hacktoberfest-accepted Recommended issues or pull requests for Hacktoberfest label Oct 16, 2020
@roclark roclark merged commit 428c497 into master Oct 16, 2020
@roclark roclark deleted the fix-nba-boxscore-home-stats branch October 16, 2020 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working hacktoberfest-accepted Recommended issues or pull requests for Hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

NBA home stats are not scraped correctly
1 participant